Skip to content

Define more GT2560 fan pins#16235

Merged
thinkyhead merged 4 commits intoMarlinFirmware:bugfix-2.0.xfrom
Vertabreak:PR2
Dec 16, 2019
Merged

Define more GT2560 fan pins#16235
thinkyhead merged 4 commits intoMarlinFirmware:bugfix-2.0.xfrom
Vertabreak:PR2

Conversation

@Vertabreak
Copy link
Copy Markdown
Contributor

@Vertabreak Vertabreak commented Dec 16, 2019

Define three FAN pins for GT2560.

@thisiskeithb
Copy link
Copy Markdown
Contributor

They were disabled in PR #13036, but no reason was given.

@Vertabreak
Copy link
Copy Markdown
Contributor Author

so can we turn them back on then?

@shitcreek
Copy link
Copy Markdown
Contributor

Let's ask @ratmole

@Vertabreak
Copy link
Copy Markdown
Contributor Author

i have several users that make use of these pins on there GT2560V3 boards it just seems strange they would be turned off.

@shitcreek
Copy link
Copy Markdown
Contributor

They weren't there before. @ratmole added them but, I guess, kept them disabled out of fear of conflicts.

@Vertabreak
Copy link
Copy Markdown
Contributor Author

i have them enabled on both my A10 & A10M right now with zero conflict.

@shitcreek
Copy link
Copy Markdown
Contributor

I understand. I'm just speculating the reasoning behind it. I'm in favor of enabling as those pins are clearly not set to anything else in the pin file.

@Vertabreak
Copy link
Copy Markdown
Contributor Author

just making it clear that i did verify it did not make anything explode on my own hardware.

@thinkyhead thinkyhead changed the title GT2560 Fans Define more GT2560 fan pins Dec 16, 2019
@thinkyhead
Copy link
Copy Markdown
Member

It may be that the current machines that ship with the GT2560 aren't using the other fans. By not defining the pins, the code ends up slightly smaller, without any reference to the unused fans. But it has been customary in Marlin to define all fan pins and compile in code for them even if they are not used.

@thinkyhead thinkyhead merged commit e0c7134 into MarlinFirmware:bugfix-2.0.x Dec 16, 2019
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 16, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 17, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
LinoBarreca pushed a commit to LinoBarreca/Marlin that referenced this pull request Dec 18, 2019
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
pull bot added a commit to anttix/Marlin that referenced this pull request Dec 18, 2019
* Define more FAN pins for GT2560 (MarlinFirmware#16235)

* Tweak E180 config

* Update product links

* [cron] Bump distribution date

* Add dev-2.1.x to 'mfinfo' script

* Move AutoBuildMarlin to its own repo
@ratmole
Copy link
Copy Markdown
Contributor

ratmole commented Jan 8, 2020

Let's ask @ratmole

had them disabled because they where not used on my printer :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants